Skip to content

Update process definition history time to live#81

Open
hauptmedia wants to merge 1 commit into
mainfrom
codex/process-definition-history-ttl
Open

Update process definition history time to live#81
hauptmedia wants to merge 1 commit into
mainfrom
codex/process-definition-history-ttl

Conversation

@hauptmedia

Copy link
Copy Markdown

Summary

  • add a Process Definition REST helper for PUT /process-definition/{id}/history-time-to-live
  • show and edit historyTimeToLive from the process definition overview
  • support clearing the value by submitting an empty field, which sends historyTimeToLive: null

Reference

Verification

  • node -e 'const fs=require("fs"); for (const locale of fs.readdirSync("public/locales")) JSON.parse(fs.readFileSync("public/locales/"+locale+"/translation.json", "utf8")); console.log("locale JSON ok")'
  • npx eslint src/pages/Processes.jsx src/pages/Processes.test.jsx src/api/resources/process_definition.js src/api/resources/process_definition.test.js src/state.js
  • npx vitest run src/api/resources/process_definition.test.js src/pages/Processes.test.jsx
  • npx vitest run --maxWorkers=4
  • npm run build
  • Browser smoke at http://127.0.0.1:5175/processes/proc:1

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
38.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant